Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Two column layout for bulk paper wallets #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gaff
Copy link

@Gaff Gaff commented Nov 8, 2013

Hi,

I'd like to generate bulk wallets in such a way that I can cut away the public keys and give people the private keys without them knowing the public keys. Obviously this only makes sense with BIP 0038 encryption. The logic being that a potential thief would then not know the balance, and thus not know if it was worth their while trying to break the encryption or not.

(My long term plan would be to have one wallet with a weak password and a token amount as a sentinel, if this wallet is stolen I'll know the others are vulnerable - but this is probably beyond the scope of bitaddress.org).

I'm not a CSS expert and I concede that the mid-word break is a little ugly, but it does the job. Happy to make ammendments though - let me know!

@pointbiz
Copy link
Owner

Would adding bip38 to the bulk wallet tab solve this?

See this issue:
#26

@Gaff
Copy link
Author

Gaff commented Nov 24, 2013

Personally I don't think so. I think the QR codes are important for paper wallets. Though adding BIP38 to bulk wallets is no bad thing of course.

@@ -5682,7 +5682,7 @@
.keyarea { font-family: Courier New; height: 110px; text-align: left; position: relative; padding: 5px; }
.keyarea .public { float: left; }
.keyarea .pubaddress { display: inline-block; height: 40px; padding: 0 0 0 10px; float: left; }
.keyarea .privwif { margin: 0; float: right; text-align: right; padding: 0 20px 0 0; position: relative; }

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gh pr checkout 28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants